Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add back getAffectedBlocks to explosion events #1647

Merged

Conversation

jpenilla
Copy link
Contributor

No description provided.

@neoforged-pr-publishing
Copy link

  • Publish PR to GitHub Packages

@neoforged-compatibility-checks
Copy link

neoforged-compatibility-checks bot commented Oct 30, 2024

@jpenilla, this PR introduces breaking changes.
Fortunately, this project is currently accepting breaking changes, but if they are not intentional, please revert them.
Last checked commit: 5430860cdf5e366c5f028280a5aff13b2f828a46.

neoforge (:neoforge)

  • net/neoforged/neoforge/event/level/ExplosionEvent$Detonate
    • <init>(Lnet/minecraft/world/level/Level;Lnet/minecraft/world/level/ServerExplosion;Ljava/util/List;)V: ❗ API method was removed
  • net/neoforged/neoforge/event/level/ExplosionKnockbackEvent
    • <init>(Lnet/minecraft/world/level/Level;Lnet/minecraft/world/level/ServerExplosion;Lnet/minecraft/world/entity/Entity;Lnet/minecraft/world/phys/Vec3;)V: ❗ API method was removed
  • net/neoforged/neoforge/event/EventHooks
    • onExplosionDetonate(Lnet/minecraft/world/level/Level;Lnet/minecraft/world/level/ServerExplosion;Ljava/util/List;D)V: ❗ API method was removed
    • getExplosionKnockback(Lnet/minecraft/world/level/Level;Lnet/minecraft/world/level/ServerExplosion;Lnet/minecraft/world/entity/Entity;Lnet/minecraft/world/phys/Vec3;)Lnet/minecraft/world/phys/Vec3;: ❗ API method was removed

@Technici4n Technici4n requested a review from XFactHD November 1, 2024 09:19
@Shadows-of-Fire Shadows-of-Fire merged commit 1be3110 into neoforged:1.21.x Nov 8, 2024
6 checks passed
@neoforged-releases
Copy link

🚀 This PR has been released as NeoForge version 21.3.12-beta.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants